Skip to content

Add optional Argo CD directory recurse support#26

Merged
hypesystem merged 3 commits into
os2ai:developfrom
felipesdacosta:add-directory-recurse-support
May 20, 2026
Merged

Add optional Argo CD directory recurse support#26
hypesystem merged 3 commits into
os2ai:developfrom
felipesdacosta:add-directory-recurse-support

Conversation

@felipesdacosta
Copy link
Copy Markdown
Contributor

Adds optional support for rendering spec.source.directory.recurse in generated Argo CD Application resources.

This allows an app to enable recursive directory discovery by setting directoryRecurse: true in applications/argo-cd-resources/values.yaml. When the value is not defined, the rendered manifests remain unchanged.

Changes

  • Added optional directoryRecurse handling in the Argo CD applications template
  • Updated CHANGELOG.md with the new unreleased change

Testing

Tested with:

helm template argo-cd-resources applications/argo-cd-resources

Also temporarily enabled directoryRecurse: true for doc-ingestion and verified that it renders directory.recurse: true.

Related issue

Fixes #25

Reviewer

@hypesystem

Copy link
Copy Markdown
Contributor

@hypesystem hypesystem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me!

It needs a rebase with develop before we can merge, but otherwise I'm ready to press the button!

- {{ . }}
{{- end }}
{{- end }}
{{- end }} No newline at end of file
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tiny: would like to preserve newline at end of file

@felipesdacosta felipesdacosta force-pushed the add-directory-recurse-support branch from 443ed0b to 05e27a3 Compare May 20, 2026 09:38
@felipesdacosta
Copy link
Copy Markdown
Contributor Author

This looks good to me!

It needs a rebase with develop before we can merge, but otherwise I'm ready to press the button!

Thanks! I’ve rebased with develop and fixed the changelog conflict. Should be ready for review/merge now.

@hypesystem hypesystem merged commit f9e2733 into os2ai:develop May 20, 2026
1 check passed
@hypesystem
Copy link
Copy Markdown
Contributor

Nice, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support directory.recurse for plain Argo CD applications

2 participants